[12.0] l10n_br_nfe: Arruma importação do campo xFant #1945
Conversation
|
/ocabot merge minor |
|
On my way to merge this fine PR! |
|
@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-1945-by-rvalyi-bump-minor. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
/ocabot merge minor |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
@marcelsavegnago your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-1945-by-marcelsavegnago-bump-minor. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
/ocabot merge minor |
|
This PR looks fantastic, let's merge it! |
|
@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-1945-by-rvalyi-bump-minor. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
/ocabot merge patch |
|
On my way to merge this fine PR! |
|
@renatonlima your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-1945-by-renatonlima-bump-patch. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
@hirokibastos Eu tentei mesclar esse PR, mas como houve uma atualização no pre-commit, você vai precisar fazer um rebase depois que o PR #1950 for mesclado |
Rebase feito |
|
/ocabot merge minor |
|
This PR looks fantastic, let's merge it! |
|
It looks like something changed on |
|
@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-1945-by-rvalyi-bump-minor. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
/ocabot merge minor |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
It looks like something changed on |
|
@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-1945-by-rvalyi-bump-minor. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
Eu acabei de cancelar porque tinha o merge do #1957 rodando e dava de novo |
hirokibastos
force-pushed
the
fix/xFant_import
branch
from
50a483a to
22ede8d
Compare
|
/ocabot merge minor |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at f6f0469. Thanks a lot for contributing to OCA. ❤️ |
Divisão de #1701
Arruma importação do campo nfe40_xFant em res_company e res_partner.
Em res_company, em alguns casos, o campo name era preenchido primeiramente com o valor de nfe40_xNome, e caso este não existisse, com o valor de nfe40_xFant.
Esta relação esta invertida, visto que o mapeamento correto é nfe40_xFant -> name e nfe40_xNome -> legal_name
Em res_partner apenas não existia um campo no qual realizar a importação, então ele foi adicionado.